Skip to content

Clarify that backport PR needs to include GH- from master#392

Merged
Mariatta merged 1 commit into
masterfrom
clarify-backport-pr-title
Jul 21, 2018
Merged

Clarify that backport PR needs to include GH- from master#392
Mariatta merged 1 commit into
masterfrom
clarify-backport-pr-title

Conversation

@Mariatta

Copy link
Copy Markdown
Member

Backport PR needs to be in the form of [X.Y] title (GH-NNNN), this is the pattern that bedevere uses:

https://github.com/python/bedevere/blob/master/bedevere/backport.py#L11

Adding a bookmark link, so I can reference it from a new status check proposed in python/core-workflow#211 (comment)

Mariatta added a commit to python/bedevere that referenced this pull request Jun 24, 2018
If a PR is made not against master, check the PR title.
If the PR title does not match the backport pattern,
post a failure status.

Two ways to resolve:
- reopen the PR against master
- Update the PR title accordingly

Depends on python/devguide#392

Closes python/core-workflow#211
Closes #111

@willingc willingc left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Mariatta Looks good. Would you mind rebasing and merging? Thanks 👍

@Mariatta Mariatta force-pushed the clarify-backport-pr-title branch from bedd994 to 6236a1e Compare July 13, 2018 18:04
@Mariatta Mariatta merged commit 2ae9f78 into master Jul 21, 2018
@Mariatta Mariatta deleted the clarify-backport-pr-title branch July 21, 2018 02:08
@Mariatta

Copy link
Copy Markdown
Member Author

Thanks!

Mariatta added a commit to python/bedevere that referenced this pull request Jul 21, 2018
If a PR is made not against master, check the PR title.
If the PR title does not match the backport pattern,
post a failure status.

Two ways to resolve:
- reopen the PR against master
- Update the PR title accordingly

Depends on python/devguide#392

Closes python/core-workflow#211
Closes #111
Mariatta added a commit to python/bedevere that referenced this pull request Jul 21, 2018
* Post a status check for backport PR title.

If a PR is made not against master, check the PR title.
If the PR title does not match the backport pattern,
post a failure status.

Two ways to resolve:
- reopen the PR against master
- Update the PR title accordingly

Depends on python/devguide#392

Closes python/core-workflow#211
Closes #111
AA-Turner pushed a commit to AA-Turner/devguide that referenced this pull request Jun 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants